memory_tests: update reporting#36413
Conversation
| @dataclass | ||
| class TestCase: | ||
| model_id: str | ||
| model_path: str |
There was a problem hiding this comment.
Use path class to store path instead string with native path separator and than replace should be not required.
| device: str | ||
|
|
||
| # metadata | ||
| original_model_path: str |
There was a problem hiding this comment.
Use path class to store path?
| def __init__(self, executable, ir_cache_dirs, devices, api=None, report_reference=False): | ||
| self.executable = executable | ||
| self.test_name = executable.rsplit("/", 1)[-1].removesuffix(".exe").removeprefix("test_") | ||
| self.test_name = executable.replace("\\", "/").rsplit("/", 1)[-1].removesuffix(".exe").removeprefix("test_") |
There was a problem hiding this comment.
instead replace can native path separator be use to split path to parts?
github-merge-queue
Bot
removed this pull request from the merge queue due to failed status checks
CI Doctor — Merge Queue failure on this PRPipeline: Windows (VS 2022, Python 3.11, Release) Possible remedyBoth failures are unrelated to your PR changes (
What happenedTwo independent Windows-only transient failures occurred:
This is a known recurring pattern for both failure types in the Windows merge-queue CI.
|
3 participants
Details:
AI Assistance: